Skip to content

Conversation

razor-x
Copy link
Member

@razor-x razor-x commented Jul 2, 2025

No description provided.

@razor-x razor-x force-pushed the evan/cx-417-json-response-blurbs-for-specialized-devices-need-to-show branch from 738a65c to 0242f87 Compare July 2, 2025 20:34
@razor-x razor-x force-pushed the evan/cx-417-json-response-blurbs-for-specialized-devices-need-to-show branch from 0242f87 to 5da951f Compare July 2, 2025 20:35
@mikewuu
Copy link
Contributor

mikewuu commented Jul 19, 2025

@razor-x this PR looks like it's working as intended, no? It re-uses the same filter in api-route.ts instead of the current selection for the sample, which is just selecting the first one:

api-endpoint.ts

  const sample = resource.resourceSamples[0]
  if (sample == null) return []

@mikewuu mikewuu marked this pull request as ready for review July 21, 2025 03:20
@mikewuu mikewuu requested a review from a team as a code owner July 21, 2025 03:20
@razor-x
Copy link
Member Author

razor-x commented Jul 23, 2025

@razor-x this PR looks like it's working as intended, no? It re-uses the same filter in api-route.ts instead of the current selection for the sample, which is just selecting the first one:

api-endpoint.ts

  const sample = resource.resourceSamples[0]
  if (sample == null) return []

@mikewuu the issue is that this did not filter the sample as expected, hence I must be missing something.

On this page the sample should be the thermostat not the lock https://docs.seam.co/latest/api/thermostats/list
image

* include parent metadata

* ci: Generate docs

* update to only select  parent metadata

* ci: Generate docs

* fallback endpoint sample

* ci: Generate docs

---------

Co-authored-by: Seam Bot <[email protected]>
@mikewuu mikewuu reopened this Jul 30, 2025
@mikewuu mikewuu requested a review from a team as a code owner July 30, 2025 13:35
@mikewuu mikewuu merged commit b04fc72 into main Jul 30, 2025
14 checks passed
@mikewuu mikewuu deleted the evan/cx-417-json-response-blurbs-for-specialized-devices-need-to-show branch July 30, 2025 13:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants